Clean up completed and won't-fix plans - #163
Conversation
df43f61 to
6f07166
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #163 +/- ##
=======================================
Coverage 88.03% 88.03%
=======================================
Files 110 110
Lines 14106 14106
=======================================
Hits 12418 12418
Misses 1228 1228
Partials 460 460 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This PR aims to declutter the active plan/ directory by archiving completed plan documents (50–88 range) and updating PLAN.md so the active roadmap table no longer lists those completed items.
Changes:
- Removed multiple completed plan documents from
plan/(intended to be archived). - Updated
PLAN.mdto remove the archived/completed plans from the active plans table.
Reviewed changes
Copilot reviewed 25 out of 25 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| plan/50_redundancy-duplication-detection.md | Removed completed plan from active plan/ directory (intended for archive). |
| plan/51_section-level-size-limits.md | Removed completed plan from active plan/ directory (intended for archive). |
| plan/62_corpus-acquisition.md | Removed completed plan from active plan/ directory (intended for archive). |
| plan/64_spike-go-native-linear-classifier.md | Removed completed plan from active plan/ directory (intended for archive). |
| plan/66_unified-conciseness-score.md | Removed completed plan from active plan/ directory (intended for archive). |
| plan/69_include-enhancements.md | Removed completed plan from active plan/ directory (intended for archive). |
| plan/73_unify-template-directives.md | Removed completed plan from active plan/ directory (intended for archive). |
| plan/74_directive-guide.md | Removed completed plan from active plan/ directory (intended for archive). |
| plan/75_single-brace-placeholders.md | Removed completed plan from active plan/ directory (intended for archive). |
| plan/76_rename-misleading-params.md | Removed completed plan from active plan/ directory (intended for archive). |
| plan/77_template-composition-and-cycles.md | Removed completed plan from active plan/ directory (intended for archive). |
| plan/79_nested-frontmatter-access.md | Removed completed plan from active plan/ directory (intended for archive). |
| plan/80_terminal-recording-readme.md | Removed completed plan from active plan/ directory (intended for archive). |
| plan/81_oom-file-size-limit.md | Removed completed plan from active plan/ directory (intended for archive). |
| plan/82_yaml-billion-laughs.md | Removed completed plan from active plan/ directory (intended for archive). |
| plan/87_markdown-flavor-github-alerts.md | Removed completed plan from active plan/ directory (intended for archive). |
| plan/88_toc-directive-migration.md | Removed completed plan from active plan/ directory (intended for archive). |
| PLAN.md | Removed rows for archived/completed plans from the active catalog table. |
|
🟢 Merge Queue — picked up This PR is in the queue and will be batched with other Next: No action needed — you'll get another comment when CI starts on the batch. View merge queue run. |
|
This PR could not be merged into the batch branch without conflicts with Next: Rebase onto or merge |
Remove 17 completed plans (status: ✅) and 5 won't fix plans (status: ⛔). Update PLAN.md table via mdsmith to show only 11 active plans. https://claude.ai/code/session_01NzqvY9FmwiSXzyGf6PdVnz
- Remove plan 53 reference from metrics-tradeoffs.md (won't fix) - Update plan 50 reference in plan 91 to avoid broken link - Fix line length issue in plan 91 https://claude.ai/code/session_01NzqvY9FmwiSXzyGf6PdVnz
4c50abd to
f5d15ca
Compare
|
@jeduden Rebased onto main, conflicts resolved, and pushed. Ready to re-queue for merge. Generated by Claude Code |
|
🟢 Merge Queue — picked up This PR is in the queue and will be batched with other Next: No action needed — you'll get another comment when CI starts on the batch. View merge queue run. |
Update generated catalog section after rebase. https://claude.ai/code/session_01NzqvY9FmwiSXzyGf6PdVnz
|
🔵 Merge Queue — CI running Merged into batch branch Next: No action needed — you'll be notified when CI completes. |
|
❌ Merge Queue — CI failed The batch CI run failed with this PR in it. Next: Fix the failure, push updates, then re-add the |
There was a problem hiding this comment.
Pull request overview
This PR prunes the roadmap by removing completed (✅) and won’t-fix (⛔) plan documents from plan/, and updates the remaining roadmap/index content so in-repo references don’t point at deleted plans.
Changes:
- Deleted 22 plan documents from
plan/(17 completed, 5 won’t-fix). - Updated
PLAN.mdto list only the remaining active/in-progress plans. - Updated documentation/plan text to remove references to deleted plan files.
Reviewed changes
Copilot reviewed 25 out of 25 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| plan/91_mds037-skip-generated-sections.md | Removes the cross-plan link to deleted plan 50 (but still needs summary text aligned). |
| plan/88_toc-directive-migration.md | Deleted completed plan doc. |
| plan/87_markdown-flavor-github-alerts.md | Deleted completed plan doc. |
| plan/82_yaml-billion-laughs.md | Deleted completed plan doc. |
| plan/81_oom-file-size-limit.md | Deleted completed plan doc. |
| plan/80_terminal-recording-readme.md | Deleted completed plan doc. |
| plan/79_nested-frontmatter-access.md | Deleted completed plan doc. |
| plan/77_template-composition-and-cycles.md | Deleted completed plan doc. |
| plan/76_rename-misleading-params.md | Deleted completed plan doc. |
| plan/75_single-brace-placeholders.md | Deleted completed plan doc. |
| plan/74_directive-guide.md | Deleted completed plan doc. |
| plan/73_unify-template-directives.md | Deleted completed plan doc. |
| plan/69_include-enhancements.md | Deleted completed plan doc. |
| plan/68_reorganize-docs.md | Deleted won’t-fix plan doc. |
| plan/66_unified-conciseness-score.md | Deleted completed plan doc. |
| plan/64_spike-go-native-linear-classifier.md | Deleted completed plan doc. |
| plan/62_corpus-acquisition.md | Deleted completed plan doc. |
| plan/58_classifier-model-selection-and-embedding.md | Deleted won’t-fix plan doc. |
| plan/56_spike-ollama-weasel-detection.md | Deleted won’t-fix plan doc. |
| plan/54_metrics-guide-tradeoffs.md | Deleted won’t-fix plan doc. |
| plan/53_conciseness-scoring.md | Deleted won’t-fix plan doc. |
| plan/51_section-level-size-limits.md | Deleted completed plan doc. |
| plan/50_redundancy-duplication-detection.md | Deleted completed plan doc. |
| docs/guides/metrics-tradeoffs.md | Removes the planned-metrics table; currently leaves inconsistent “plan 53” references that should be reconciled. |
| PLAN.md | Updates the generated plans catalog to only include remaining plan docs. |
- Update metrics-tradeoffs.md to remove plan 53 reference - Update plan 91 summary to remove plan 50 reference https://claude.ai/code/session_01NzqvY9FmwiSXzyGf6PdVnz
|
Addressed Copilot review comments:
All references to deleted plans have been cleaned up. Ready to re-add queue label for merge. Generated by Claude Code |
Summary
Remove 17 completed plans (status: ✅) and 5 won't-fix plans (status: ⛔) from the active planning directory. These documents are no longer part of the roadmap.
Changes
Deleted from
plan/: 22 plan documentsUpdated
PLAN.md: Removed rows for deleted plans, now showing only 11 active plansFixed broken links: Removed references to deleted plans in
docs/guides/metrics-tradeoffs.mdandplan/91_mds037-skip-generated-sections.mdResult
The plan directory now contains only active and in-progress plans, reducing cognitive load and keeping the roadmap focused on current work.
https://claude.ai/code/session_01NzqvY9FmwiSXzyGf6PdVnz